Skip to content

[beta] Stabilize unions with Copy fields and no destructor#42321

Closed
brson wants to merge 2 commits into
rust-lang:betafrom
brson:beta-next
Closed

[beta] Stabilize unions with Copy fields and no destructor#42321
brson wants to merge 2 commits into
rust-lang:betafrom
brson:beta-next

Conversation

@brson

@brson brson commented May 31, 2017

Copy link
Copy Markdown
Contributor

@rust-highfive

Copy link
Copy Markdown
Contributor

warning Warning warning

  • Pull requests are usually filed against the master branch for this repo, but this one is against beta. Please double check that you specified the right target!

@brson

brson commented May 31, 2017

Copy link
Copy Markdown
Contributor Author

Testing locally while we try to land via bors.

@aturon

aturon commented May 31, 2017

Copy link
Copy Markdown
Contributor

@bors: r+

@bors

bors commented May 31, 2017

Copy link
Copy Markdown
Collaborator

📌 Commit b112c27 has been approved by aturon

@Mark-Simulacrum

Copy link
Copy Markdown
Member

@bors p=10 (beta backport with short time remaining)

@petrochenkov

Copy link
Copy Markdown
Contributor

@brson @aturon
You may want to backport #42083 as well, they were supposed to land together.

@bors

bors commented May 31, 2017

Copy link
Copy Markdown
Collaborator

⌛ Testing commit b112c27 with merge 6ed6494...

@bors

bors commented May 31, 2017

Copy link
Copy Markdown
Collaborator

💔 Test failed - status-travis

@petrochenkov

Copy link
Copy Markdown
Contributor

I've submitted #42328 including #42083 and fixing discrepancies between beta and nightly causing the Travis failure.

@brson

brson commented May 31, 2017

Copy link
Copy Markdown
Contributor Author

Closing in favor of #42328

@brson brson closed this May 31, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants